-
Notifications
You must be signed in to change notification settings - Fork 240
fix: update to use pyhive function to get compatible sasl #1095
base: main
Are you sure you want to change the base?
Conversation
|
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: nhat.nguyent.
|
9769fd2 to
6eb39bf
Compare
|
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @ntnhaatj |
6eb39bf to
8b3380e
Compare
|
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @ntnhaatj |
|
Hi @McKnight-42 @mikealfare , any update for this PR? |
|
This PR has been marked as Stale because it has been open with no activity as of late. If you would like the PR to remain open, please comment on the PR or else it will be closed in 7 days. |
|
Hello. Will this PR be included in the official code? In our case, it solved a connectivity issue for a major banking client in a Cloudera ecosystem, with Kerberos authentication and a Livy Thrift server. |
resolves dbt-labs/dbt-adapters#475
docs dbt-labs/docs.getdbt.com/#
Problem
Current SASL factory for
TSaslClientTransportusing SASLClient frompure-saslwhich is incompatible withpyhiveSolution
Pyhive provides a wrapper PureSASLClient which implement some missing methods to interact with
thrift-sasl.This MR update to use
get_installed_saslof pyhive to decide which sasl client should be use in current environment.Checklist